Skip to content

feat(incidents): Add is_metric_subscription_allowed; use it#112241

Merged
kcons merged 4 commits intomasterfrom
kcons/onering
Apr 6, 2026
Merged

feat(incidents): Add is_metric_subscription_allowed; use it#112241
kcons merged 4 commits intomasterfrom
kcons/onering

Conversation

@kcons
Copy link
Copy Markdown
Member

@kcons kcons commented Apr 3, 2026

There are a couple places where we check features and datasets to determine whether a subscription is allowed to be enabled.
This defines that somewhere central and uses it in subscription processing.
Once it lands, we'll extend it to some getsentry logic.

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 3, 2026
@kcons kcons marked this pull request as ready for review April 3, 2026 23:16
@kcons kcons requested a review from a team as a code owner April 3, 2026 23:16

Returns True if allowed, False if the organization lacks the required features
(e.g. after a plan downgrade).
"""
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: maybe make a has_incidents var to reuse?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done.

@kcons kcons enabled auto-merge (squash) April 6, 2026 22:54
@kcons kcons merged commit 30e2e32 into master Apr 6, 2026
59 of 60 checks passed
@kcons kcons deleted the kcons/onering branch April 6, 2026 23:04
george-sentry pushed a commit that referenced this pull request Apr 9, 2026
There are a couple places where we check features and datasets to
determine whether a subscription is allowed to be enabled.
This defines that somewhere central and uses it in subscription
processing.
Once it lands, we'll extend it to some getsentry logic.
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 22, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants